Comments: drop port/audit tags and line citations (Windows, Dart, tooling) - #53

Comments and docs written during the Windows port and the audits refer to things a new reader can't look up:

• Tags: LAW 1…LAW 10, SLICE, spike ids (S1…S6, S6a, "the S3 recipe", "S1 belt-1"), phase ids (P6, P7, P9, P11, "pre-P9", CEF-1, CEF-2a/2b), audit ids (C1, C2, C3, H3, H7, "sev-8 audit findings"), audit finding numbers #1/#3/#9 (on GitHub these link to unrelated PRs), fix ids F-1…F-6, and PLAN §x.y.
• Line citations: main.mm:NNN, Swift:NNN, FlutterCefPlugin.swift:NNN, cef_web_controller.dart:NNN and similar. Every main.mm line number has been wrong since macOS: split cef_host's main.mm and CefProfileHost; one record per session #50 split main.mm into nine files.

This PR replaces each tag with the reason it stood for. For example, "size-gate (LAW 4)" now says that CEF keeps delivering frames at the old size after a resize, and the host's "THE LAWS" list is now a list of plain rules. Where a sentence already gave the reason, the tag is simply dropped. Line citations now name the macOS file and function (for example DoSetVisible in browser_ops.mm, failHost in FlutterCefPlugin.swift); each name was checked with grep. Citations that added nothing are gone. specs/... and SPIKES.md pointers are replaced by the reason they stood for.

PROTOCOL.md changes:

• The method table loses its Slice? and Source (Swift:NNN) columns.
• The events table loses its Source column, and the cookies table loses its main.mm column.
• The one fact the Slice column carried that still matters, that showDevTools and the ime* verbs are unverified on Windows OSR, is now a sentence.
• The prose citations now point at files and functions.
• The generated opcode section is untouched; dart run tool/protocol/generate.dart makes no change.

The PR also fixes some stale pointers found along the way:

• In PORTING.md, the table of macOS seams had main.mm line numbers and a g_surface global that no longer exists.
• PROTOCOL.md gave the wrong path for CdpRelayFilterTests.swift.

Scope: everything except packages/flutter_cef_macos/**, which is swept separately, docs/history/** and the CHANGELOGs.

Counts

With the sweep's counting regex, excluding docs/history, the CHANGELOGs and flutter_cef_macos:

• Before: 223 matching lines, of which 197 are real tags or citations.
• After: 28 matching lines, of which 2 are real (see below).
• The other 26 matches, before and after, are false positives:
  • 10 are bytes in the example's PNG app icons.
  • 16 are in code: (x64) in CONTRIBUTING, %ProgramFiles(x86)%, JS .slice(1), x1)/y0) in the popup blit, *i32)/*i64), and test values like f1), b64), protocolMismatch(host=v3).

A wider scan also catching S/P/C/H/F-n/#n/PLAN §/CEF-2x tags and bare line ranges went from about 330 lines to the handful listed below.

Left in place

• cef_host_win.cc, OnBeforePopup: a runtime SendLog string still says "post-slice on Windows" and "(macOS OpenNativeAuthPopup, main.mm:1444)". It is log output, not a comment, so this comments-only PR leaves it. It is worth a one-line follow-up.
• example/lib/multiview_probe.dart: the on-screen label 'P2-step2 probe — …' is a UI string, left for the same reason. The probe's check labels A/D/E/F stay; the file header now defines them.
• packages/flutter_cef_windows/README.md: "the original port plan and spikes are in docs/history/windows-port/" is a real pointer to the history docs.
• Legitimate words and references: "keystone" in ci.yaml, "C0 controls" (the Unicode control-character block) in cef_web_view.dart, PROTOCOL.md §n (real sections), commit 855042d, and "CEF t=11650" (a CEF forum thread).

Verification

• fvm flutter analyze is clean at the root and in example/.
• fvm flutter test: 280 passed, including the protocol parity test.
• dart run tool/protocol/generate.dart leaves the tree unchanged.
• bash -n passes on example/run_leak_soak.sh and example/run_conformance_oracle.sh.
• For all 13 changed C/C++ files, stripping comments and collapsing whitespace gives the same token stream as main. A deliberate one-token code edit (negative control) did change the stream. So no comment was left unterminated and no // swallowed code.
• The CMake, PowerShell, shell and Dart diffs change only comment lines.
• The Windows C++ is compiled by CI's windows-build job.

🤖 Generated with Claude Code